Skip to content

chore(audit): apply ganda repo audit --fix - #39

Merged
StevenTCramer merged 6 commits into
masterfrom
task/030-bring-ganda-repo-audit-clean
Nov 8, 2026
Merged

StevenTCramer merged 6 commits into
masterfrom
task/030-bring-ganda-repo-audit-clean

Conversation

@StevenTCramer

Copy link
Copy Markdown
Contributor

Summary

ganda repo audit exits 0 with no Failed row (Passed 36, Failed 0, Skipped 3). global-usings-analyzer is off under [ganda.audit] with the reason comment, and there is no root PackageReference to TimeWarp.SourceGenerators.

--fix is committed as 0eac4eb (chore(audit): apply ganda repo audit --fix). That commit keeps the Amuru 3 and Terminal bumps. Hand fixes are a936c58.

Nuru and DevCli are 4.0.0-beta.11. Nuru 4 moved ICommand, ICommandHandler, and Unit to TimeWarp.Mediator, and handlers return Task<Unit>. The dev CLI compiles on net11.0. dotnet run --file tools/dev-cli/dev.cs -- --help lists release. check-version prints Packages checked: TimeWarp.SourceGenerators and exits 1 because 1.0.0-beta.11 is already on NuGet. That exit is the release guard, not a missed package.

markdown-docs-generator.cs is 136 lines. Markdown parsing lives in markdown-docs-generator.markdown-to-xml.cs (409 lines). ./bin/dev build succeeded with 0 warnings. ./bin/dev test passed the test console and 11/11 analyzer tests. The Nuru 4 dev CLI build command also succeeded.

Task 027 is archived on master (09a725d). This walk did not run ganda kanban done or gh pr create.

CI installs the net11 RC next to net10 so dotnet run --file tools/dev-cli/dev.cs can restore Amuru 3. .NET 11 was still 11.0.0-rc.3 on 2026-11-08.

Review disposition

  • Disposition: clean. Rounds: 1. Effort 3, roster: general.
  • Final counts: bug 0, suggestion 0, nit 0 (0 open, 0 fixed, 0 wontfix).
  • Live checks by the reviewer: ganda repo audit exit 0 (Passed 36, Failed 0, Skipped 3). ./bin/dev build reported 0 warnings and 0 errors. ./bin/dev test passed 11/11. check-version lists TimeWarp.SourceGenerators. The generator split is confirmed as a pure move.
  • Follow-up (not a finding): after .NET 11 GA, change the CI SDK line 11.0.100-rc.3.26522.148 to 11.0.x.
  • Paths: review/review-framework.md, review/round-1/general.md, review/round-1/merged.md, review/disposition.md.

Test plan

ganda repo audit

Kanban task 030
Session: Grok 01a1bbe3-2e77-79f0-ae81-48feae04ec9c

The live audit still failed on glossary, hooks, the Nuru pin, required gitignore entries, vscode settings, the window icon, and pull_request branches. --fix is the baseline, including the Amuru 3 and Terminal pin moves that came with the Nuru bump.

why: task 030
Charset, the self-package analyzer downgrade, and the markdown docs split are the rows --fix cannot take. Nuru 4 moved ICommand and Unit to TimeWarp.Mediator and returns Task, and the Amuru 3 pin is net11-only, so the dev CLI and the git hooks target net11 and CI installs that SDK.

why: task 030
The audit is clean and the validation commands have been run. Task 027 is archived on master so it does not edit the same gitignore.

why: task 030
@StevenTCramer
StevenTCramer merged commit 4ae398b into master Nov 8, 2026
1 check passed
@StevenTCramer
StevenTCramer deleted the task/030-bring-ganda-repo-audit-clean branch November 8, 2026 16:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant